New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
+UPSND sent to poll activation status #10636
Conversation
@bqam-ublox, thank you for your changes. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please fix coding style (see Travis failures)
@0xc0170 coding style fixed |
@@ -192,14 +191,29 @@ bool UBLOX_AT_CellularContext::activate_profile(const char *apn, | |||
if (_at.get_last_error() == NSAPI_ERROR_OK) { | |||
// Activate, wait upto 30 seconds for the connection to be made | |||
_at.set_at_timeout(30000); | |||
_at.cmd_start("AT+UPSDA=0,3"); | |||
_at.cmd_start("AT+UPSD=" PROFILE ",6,"); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you check UPSDA
is dismissed here intentionally?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This was a mistake! Thanks for pointing out, corrected.
CI started |
Test run: SUCCESSSummary: 11 of 11 test jobs passed |
Description
This pull request updates the context activation using the +UPSND AT command. Previously, the context status was updated immediately after activating the context (without verifying if it was activated successfully).
+UPSND command responds with the status of the context activation. The status is polled every 5s with a max timeout of 180s.
Pull request type
Reviewers
Release Notes